Skip to content

PEP 817: Wheel Variants: Beyond Platform Tags #4740

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
DEKHTIARJonathan wants to merge 63 commits into
base: main
Choose a base branch
from
Open

PEP 817: Wheel Variants: Beyond Platform Tags #4740

DEKHTIARJonathan wants to merge 63 commits into from
+2,785 −0

Conversation

@DEKHTIARJonathan
Copy link

@DEKHTIARJonathan DEKHTIARJonathan commented Dec 10, 2025 •
edited by hugovk
Loading

Basic requirements (all PEP Types)

  • Read and followed PEP 1 & PEP 12
  • File created from the latest PEP template
  • PEP has next available number, & set in filename (pep-NNNN.rst), PR title (PEP 123: <Title of PEP>) and PEP header
  • Title clearly, accurately and concisely describes the content in 79 characters or less
  • Core dev/PEP editor listed as Author or Sponsor, and formally confirmed their approval
  • Author, Status (Draft), Type and Created headers filled out correctly
  • PEP-Delegate, Topic, Requires and Replaces headers completed if appropriate
  • Required sections included
    • Abstract (first section)
    • Copyright (last section; exact wording from template required)
  • Code is well-formatted (PEP 7/PEP 8) and is in code blocks, with the right lexer names if non-Python
  • PEP builds with no warnings, pre-commit checks pass and content displays as intended in the rendered HTML
  • Authors/sponsor added to .github/CODEOWNERS for the PEP

Standards Track requirements

  • PEP topic discussed in a suitable venue with general agreement that a PEP is appropriate
  • Suggested sections included (unless not applicable)
    • Motivation
    • Rationale
    • Specification
    • Backwards Compatibility
    • Security Implications
    • How to Teach This
    • Reference Implementation
    • Rejected Ideas
    • Open Issues
  • Python-Version set to valid (pre-beta) future Python version, if relevant
  • Any project stated in the PEP as supporting/endorsing/benefiting from the PEP formally confirmed such
  • Right before or after initial merging, PEP discussion thread created and linked to in Discussions-To and Post-History

CC: @mgorny @konstin @rgommers @atalman @charliermarsh @msarahan @seemethere @warsaw @dstufft @aterrel

📚 Documentation preview 📚: https://pep-previews--4740.org.readthedocs.build/pep-0817/

@DEKHTIARJonathan DEKHTIARJonathan requested a review from a team as a code owner December 10, 2025 21:54
@python-cla-bot
Copy link

python-cla-bot bot commented Dec 10, 2025 •
edited
Loading

All commit authors signed the Contributor License Agreement.

CLA signed

@hugovk
Copy link
Member

hugovk commented Dec 10, 2025 •
edited
Loading

Let's renumber this to 9999 for now, we also have #4739 clashing.

@warsaw or @dstufft Please can you confirm co-authorship?

@hugovk hugovk changed the title PEP 817: Wheel Variants: Beyond Platform Tags PEP 9999: Wheel Variants: Beyond Platform Tags Dec 10, 2025
@hugovk hugovk mentioned this pull request Dec 10, 2025
Open
27 tasks
@DEKHTIARJonathan
Copy link
Author

DEKHTIARJonathan commented Dec 10, 2025

Oh thanks @hugovk I totally didn't see that one being published within 20min :D

Let me know which number you want me to pick and I'll do the update ;)

@warsaw
Copy link
Member

warsaw commented Dec 10, 2025

@warsaw or @dstufft Please can you confirm co-authorship?

Confirmed.

@hugovk hugovk changed the title PEP 9999: Wheel Variants: Beyond Platform Tags PEP 817: Wheel Variants: Beyond Platform Tags Dec 10, 2025
@hugovk
Copy link
Member

hugovk commented Dec 10, 2025

Thanks!

@DEKHTIARJonathan You may continue with 817.

hugovk
hugovk reviewed Dec 10, 2025
View reviewed changes
@AA-Turner AA-Turner added the new-pep A new draft PEP submitted for initial review label Dec 11, 2025
AA-Turner
AA-Turner reviewed Dec 11, 2025
View reviewed changes
Copy link
Member

@AA-Turner AA-Turner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very brief review, I haven't yet read beyond the end of Motivation.

A

@jezdez
Copy link

jezdez commented Dec 11, 2025

Just a brief bystander comment: I'm so stoked to see this PEP draft published!

@hugovk hugovk mentioned this pull request Dec 11, 2025
Merged
@mgorny @AA-Turner
Apply suggestions from code review
8e3d5ea 
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@DEKHTIARJonathan
Copy link
Author

DEKHTIARJonathan commented Dec 11, 2025

Just a brief bystander comment: I'm so stoked to see this PEP draft published!

Thanks Jannis! Took some significant amount of work but we eventually got there

mgorny and others added 5 commits December 11, 2025 17:26
@mgorny
Switch to plain quotation marks
f0d43f6 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Move "definitions" to "specification" section
e16bf03 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Reflow the cupy package list
fede5ff 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@DEKHTIARJonathan @AA-Turner
Update peps/pep-0817/appendix-variant-metadata-json-schema.rst
c68242d 
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@DEKHTIARJonathan
[PEP Review] JSON Schema Indent switched to 2
64f32ed
@mgorny
Replace the benchmark chart with SVG
78ff40b 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Compact the SVG
a31b68b 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Reflow text
73753b9 
Reflow the text to restore correct text width after all the inline
changes and applied suggestions.

Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Replace typographic apostrophes
162ec7b 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove double spaces introduced while reflowing
0064073 
Remove accidental double spaces that vim's `gq` introduced while I was
reflowing the text.  Thanks to @konstin for noticing.

Signed-off-by: Michał Górny <mgorny@quansight.com>
seemethere
seemethere reviewed Dec 16, 2025
View reviewed changes
peps/pep-0817.rst
enabling the optional provider or selecting the variant explicitly.


Package ABI matching
Copy link

@seemethere seemethere Dec 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super excited about this one by the way :D

mgorny and others added 30 commits December 30, 2025 15:28
@mgorny
Add SLEAP quote
f48baa4 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Move "out-of-scope features" to "rejected ideas"
1d44653 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Update cupy package list, and clarify
1062280 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Revert "Move "out-of-scope features" to "rejected ideas""
7332090 
This reverts commit 261704f.

Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Move glossary and overview into Rationale
fffefe0 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove speccy parts from "Overview" rationale
a54fc2a 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove "summary of changes" for now
5a512e5 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Highlight the added filename part
7794674 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Avoid forward-linking from rationale to spec
72d4f3f 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Expand variant properties in rationale
dc7b63b 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Better explain providers in "Rationale" section
5748810 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Readd info on using providers to the spec
62be783 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Explain metadata a bit in rationale
03a51cc 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Clarify provider part of spec
e5f0f9b 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Explain ABI variant provider in the rationale
e17f3f0 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Rewording of confusing bits
2f80033 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove duplicate "publishes builds" verb
5d9c6db 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny @rgommers
Apply suggestions from code review
f6a132e 
Co-authored-by: Ralf Gommers <ralf.gommers@gmail.com>
@mgorny
project tree → source tree
2f7b7da 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
regular wheel → non-variant wheel
872213f 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Loosen and clarify verification at install time
c66ecad 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Make respecting enable-if mandatory
80300c1 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Limit recommendations to install-time providers
a60332b 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove "after verifying their trustworthiness"
8bc3b51 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Try to clarify build-time validation
d590d04 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Update links to where security is covered now
c98613f 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove duplicate/non-spec content in Providers spec
5257463 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Move environment and pinning bits to rationale
5c43293 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Mention metadata bits in overview
a2d653d 
Signed-off-by: Michał Górny <mgorny@quansight.com>
@mgorny
Remove some duplication in "Provider information"
2ce4d07 
Environment marker usage and optionality is described in "Providers"
section. Let's limit "Provider information" only to technical field
details themselves.

Signed-off-by: Michał Górny <mgorny@quansight.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hugovk hugovk hugovk left review comments

@willingc willingc willingc left review comments

@AA-Turner AA-Turner AA-Turner left review comments

@emmatyping emmatyping emmatyping left review comments

+4 more reviewers

@rgommers rgommers rgommers left review comments

@mgorny mgorny mgorny left review comments

@seemethere seemethere seemethere left review comments

@konstin konstin konstin left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

new-pep A new draft PEP submitted for initial review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.